Skip to content

[18.0-fr6] Set ImagePullPolicy to IfNotPresent on OpenStackClient pod#1953

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr6from
openshift-cherrypick-robot:cherry-pick-1949-to-18.0-fr6
Jun 24, 2026
Merged

[18.0-fr6] Set ImagePullPolicy to IfNotPresent on OpenStackClient pod#1953
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18.0-fr6from
openshift-cherrypick-robot:cherry-pick-1949-to-18.0-fr6

Conversation

@openshift-cherrypick-robot

@openshift-cherrypick-robot openshift-cherrypick-robot commented Jun 23, 2026

Copy link
Copy Markdown

This is an automated cherry-pick of #1949

/assign abays

@abays @claude
Set ImagePullPolicy to IfNotPresent on OpenStackClient pod
95be94c 
The OpenStackClient pod is created as a raw corev1.Pod directly,
bypassing lib-common workload helpers where SetPullPolicyDefaults
is applied. When ImagePullPolicy is not explicitly set and the
image tag is "latest" or unset, Kubernetes defaults the policy to
Always. Explicitly set PullIfNotPresent to avoid unnecessary image
pulls.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Jun 23, 2026
Merged
@openshift-ci openshift-ci Bot requested review from rabi and slagle June 23, 2026 14:01
@github-actions

github-actions Bot commented Jun 23, 2026

Copy link
Copy Markdown

OpenStackControlPlane CRD Size Report

Metric Value
CRD JSON size 349969 bytes (342KB)
Base branch size 349969 bytes
Change +0.00%
Status yellow — growing
Threshold reference
Color Range Meaning
🟢 green < 300KB Comfortable
🟡 yellow 300–400KB Growing
🟠 orange 400–750KB Concerning
🔴 red > 750KB Approaching 1.5MB etcd limit (cut in half to allow space for update)

abays
abays approved these changes Jun 23, 2026
View reviewed changes

@abays abays left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Jun 23, 2026
@openshift-ci

openshift-ci Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, openshift-cherrypick-robot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@abays

abays commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

/test openstack-operator-build-deploy-kuttl-4-18

@abays

abays commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

@openshift-cherrypick-robot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:
Test name Commit Details Required Rerun command
ci/prow/openstack-operator-build-deploy-kuttl-4-18 95be94c link true /test openstack-operator-build-deploy-kuttl-4-18

Full PR test history. Your PR dashboard.
Details

Cert rotation KUTTL test has failed twice in two attempts.  Seems like it might be a legitimate problem.

@abays

abays commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

/test openstack-operator-build-deploy-kuttl-4-18

@stuggi

stuggi commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Failed on must-gather

@stuggi

stuggi commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

/retest

1 similar comment
@stuggi

stuggi commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

/retest

@openshift-merge-bot openshift-merge-bot Bot merged commit c4438a5 into openstack-k8s-operators:18.0-fr6 Jun 24, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abays abays abays approved these changes
@rabi rabi Awaiting requested review from rabi
@slagle slagle Awaiting requested review from slagle

Assignees

@abays abays

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@openshift-cherrypick-robot @abays @stuggi